Skip to content

ensure form-decode-str encoding is a String#23

Open
mccraigmccraig wants to merge 1 commit intoring-clojure:masterfrom
yapsterapp:require-character-encoding
Open

ensure form-decode-str encoding is a String#23
mccraigmccraig wants to merge 1 commit intoring-clojure:masterfrom
yapsterapp:require-character-encoding

Conversation

@mccraigmccraig
Copy link

throws an Exception if the encoding arg given to the arity-2 version of
form-decode-str is not a String value, because a nil encoding will
cause a surprising {nil nil} parse result

addresses issue: #22

@weavejester
Copy link
Member

What's your reasoning for using an ex-info rather than a precondition?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants